Skip to content

Chip revision fix #704

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 2 commits into from
Oct 8, 2017
Merged

Chip revision fix #704

merged 2 commits into from
Oct 8, 2017

Conversation

Testato
Copy link
Contributor

@Testato Testato commented Oct 7, 2017

igrr
igrr reviewed Oct 7, 2017
View reviewed changes
Copy link
Member

@igrr igrr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How about using esp_chip_info function to get the revision?

@Testato
Copy link
Contributor Author

Testato commented Oct 7, 2017 •
edited
Loading

yep,
added, tested, it work

@me-no-dev me-no-dev merged commit 8dc0c86 into espressif:master Oct 8, 2017
@me-no-dev
Copy link
Member

Cool :) merged!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@igrr igrr igrr left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Testato @me-no-dev @igrr